Skip to content

Conversation

@speige
Copy link
Contributor

@speige speige commented Dec 17, 2024

No description provided.

@speige speige force-pushed the ParserUnaryMinusOverflow branch from feb2d1a to 766b719 Compare December 17, 2024 16:42
@Drake53
Copy link
Owner

Drake53 commented Dec 19, 2024

Thanks for the bugfix PR, but the JASS library implementation in the master branch is kind of deprecated, because I made a new implementation on the vjass branch (which I basically use as a development branch at the moment). The new implementation does not attempt to evaluate the value of the expression so it does not have this issue.

The new implementation is closer to roslyn/C# syntax, and it supports trivia so you can make changes to the syntax tree without losing whitespace information. Obviously this is a breaking change and I never got around to rewriting all the code which depends on the JASS library (including the decompiler project which I see you also made a PR for).

@speige speige closed this Sep 6, 2025
@speige speige deleted the ParserUnaryMinusOverflow branch September 6, 2025 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants